From: Owen W. Taylor <otaylor@fishsoup.net>
Date: Sat, 8 May 2010 16:16:54 +0000 (-0400)
Subject: Use a St.Bin rather than a St.BoxLayout for calendar popup
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~10122
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=5111edb80b8714f4cf176f1778541d5766242d07;p=gnome-shell.git

Use a St.Bin rather than a St.BoxLayout for calendar popup

Using a horizontal St.BoxLayout for calendar container forces
width-for-height layout on the St.Table child. Since St.Table
is naturally width-for-height, this can trigger bugs and is,
at best, a bit ineffecient. Use a St.Bin instead since we don't
need any BoxLayout features.

https://bugzilla.gnome.org/show_bug.cgi?id=618104
---

diff --git a/js/ui/panel.js b/js/ui/panel.js
index 34f5696e4..632016b25 100644
--- a/js/ui/panel.js
+++ b/js/ui/panel.js
@@ -714,10 +714,10 @@ CalendarPopup.prototype = {
     _init: function() {
         let panelActor = Main.panel.actor;
 
-        this.actor = new St.BoxLayout({ name: 'calendarPopup' });
+        this.actor = new St.Bin({ name: 'calendarPopup' });
 
         this.calendar = new Calendar.Calendar();
-        this.actor.add(this.calendar.actor);
+        this.actor.set_child(this.calendar.actor);
 
         Main.chrome.addActor(this.actor, { visibleInOverview: true,
                                            affectsStruts: false });